Skip to content

WIP: Livewire the importer upload (move away from blueimp) #13793

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

snipe
Copy link
Member

@snipe snipe commented Oct 25, 2023

This removed the blueimp dependency - which we were apparently only using in a single vestigial place. This is not done yet - there is no controller waiting to do anything with the file yet, and there are definitely some additional tweaks with regard to styling the progress bar, etc, but I at least wanted to put this out there for now.

@snipe snipe marked this pull request as draft October 25, 2023 12:24
@what-the-diff
Copy link

what-the-diff bot commented Oct 25, 2023

PR Summary

  • Update in app/Http/Livewire/Importer.php
    The code has been tweaked to enhance the file uploading process.
  • Change in package.json
    A file upload helper library is no longer needed and thus has been removed.
  • Refinement in public/css/dist/all.css
    Certain old file upload related stylistic rules have been done away with.
  • Adjustment to public/mix-manifest.json
    This deals with the method of caching files for performance. The unique identifiers have been updated for some files, ensuring fresh copies are served to users.
  • Modification in resources/views/layouts/default.blade.php
    A new style tag was introduced for further aesthetic development.
  • Improvement in resources/views/livewire/importer.blade.php
    The user interface to upload files has been updated. Unneeded code for the old upload system was removed.
  • Update in webpack.mix.js
    The old and now unnecessary file upload dependencies were removed from the compilation configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant